-
Notifications
You must be signed in to change notification settings - Fork 38.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Ingress doc clarifications #16932
Ingress doc clarifications #16932
Conversation
|
||
Before you can receiver traffic through the GCE L7 Loadbalancer Controller you need: | ||
* A Working Kubernetes 1.1 cluster | ||
* At least 1 Kubernets [NodePort service](../../../../docs/user-guide/services.md#type-nodeport) (this is the endpoint for your Ingress) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/Kubernets/Kubernetes
Labelling this PR as size/S |
GCE e2e test build/test passed for commit a7752bed9db084c914bb8fe48fc78521ad471f75. |
lgtm, thanks |
Added a line about HTTPS, re-adding lgtm to keep merge bot happy |
GCE e2e test build/test passed for commit d4d0addb6e10569df783f68be61a36689e109a14. |
GCE e2e build/test failed for commit 10d1d4c. |
@k8s-bot test this please |
GCE e2e build/test failed for commit 10d1d4c. |
@k8s-bot test this please |
GCE e2e test build/test passed for commit 10d1d4c. |
@k8s-bot test this [submit-queue is verifying that this PR is safe to merge] |
GCE e2e build/test failed for commit 10d1d4c. |
Jenkins flakes are unrelated, shippable and unittests passed, merging manually for release |
…6932-upstream-release-1.1 Auto commit by PR queue bot
…pick-of-#16932-upstream-release-1.1 Auto commit by PR queue bot
…pick-of-#16932-upstream-release-1.1 Auto commit by PR queue bot
Last minute doc clarifications. Assigning to Jeff since he has context, feel free to re-assign.